[dotnet] only embed the git revision in SourceLink metadata when stamping - #18004

• Gates SourceLink git revision embedding behind Bazel stamping.
• Uses HEAD metadata for unstamped builds, restoring remote cache reuse.
• Preserves exact commit links for stamped release artifacts.

graph TD
  A["Bazel Build"] --> B["Stamp Config"] --> C["SourceLink Rule"] --> D{"Stamped?"}
  D -->|Yes| E["Git Revision"] --> G["SourceLink JSON"] --> H["C# Assemblies"]
  D -->|No| F["HEAD Fallback"] --> G

Loading

The selected approach is appropriate: it preserves revision-accurate SourceLink data for stamped releases while removing workspace-status dependencies from ordinary builds. A stable HEAD fallback retains SourceLink functionality without forcing assembly and test cache invalidation, and callers can still override the stamp attribute explicitly.

Bug fix (1) +29 / -1

sourcelink.bzlAvoid workspace revision inputs for unstamped SourceLink builds +29/-1

Avoid workspace revision inputs for unstamped SourceLink builds

• Adds a stamp attribute and macro that defaults to the global Bazel stamping flag. Unstamped builds write stable HEAD-based metadata directly, while stamped builds continue reading the exact revision from workspace status.

dotnet/private/sourcelink.bzl

Other (1) +6 / -0

BUILD.bazelExpose Bazel stamping as a private .NET configuration target +6/-0

Expose Bazel stamping as a private .NET configuration target

• Adds a package-visible config setting that detects when Bazel stamping is enabled. The SourceLink macro uses this setting to derive its default stamp behavior.

dotnet/private/BUILD.bazel

Description

csharp_sourcelink_library now selects between HEAD metadata and workspace-status revision
metadata without a focused automated test. A later change to either stamping branch could make every
.NET assembly and test depend on the current revision again or omit the revision from published
metadata without detection.

Code

dotnet/private/sourcelink.bzl[R15-18]

+    if not ctx.attr.stamp:
+        ctx.actions.write(
+            output = output,
+            content = '{{"documents":{{"*":"{repo}/raw/HEAD/*"}}}}\n'.format(

Evidence

Compliance rule 5 requires focused regression coverage for changed behavior. The cited
implementation introduces distinct stamped and unstamped metadata paths, but the PR adds no test
exercising either path.

AGENTS.md: Add Focused Tests for Behavioral Changes and Fixes
dotnet/private/sourcelink.bzl[13-30]

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
Add focused regression coverage for SourceLink metadata generation in stamped and unstamped builds.

## Issue Context
The unstamped path must generate a `HEAD` URL without reading workspace status, while the stamped path must embed the stable Git revision and retain its existing fallback behavior.

## Fix Focus Areas
- dotnet/private/sourcelink.bzl[13-30]
- dotnet/private/sourcelink.bzl[102-125]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

💡 Did you know, you can tweak Display preferences with a live preview to see your comment before it ships

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗